- Use consistent camelCase in error messages (anchorLine, maxLines instead of snake_case)
- Accept both camelCase and snake_case for indentation config in NativeToolCallParser
- Continue counting total lines after hitting limit in readSlice() for accurate metadata
- Return empty content instead of throwing when offset exceeds file length
- Disable strict mode in tool schema when partial reads enabled to allow truly optional params
- Update tests to match new behavior

…data

…indentation mode

When totalLinesInFile is 0, there are no lines before the anchor point,
so hasMoreBefore must be false regardless of the offset/anchorLine value.

The limit parameter was defined in FileEntry but intentionally ignored by
ReadFileTool since the line limit is controlled by the maxReadFileLine
setting. This removes the unused type field and parsing code to align
the types with actual behavior.

- Remove limit from FileEntry interface in tool-params.ts
- Remove limit parsing from NativeToolCallParser.convertFileEntries()
- Remove limit from FileResult interface in ReadFileTool.ts
- Update comment to remove obsolete limit mention

…lOptions

…leLine setting

Addresses review comment about FileEntry.limit being defined but ignored.
The limit parameter was intentionally not exposed to models - it's controlled
by the maxReadFileLine user setting instead. Updated the comment in
NativeToolCallParser.ts to clarify this design decision.

Updates tests to match the new API that replaces line_ranges with:
- offset: 1-indexed starting line number
- mode: 'slice' or 'indentation' reading mode
- indentation: configuration for indentation mode

Also updates strict mode expectations since partialReadsEnabled (default true)
now sets strict to false to allow optional parameters.

…test

The ReadFileTool's execute method requires additional Task methods that were
not mocked:
- sayAndCreateMissingParamError: called when files array is empty/undefined
- cwd: used for path resolution
- apiConfiguration: used for protocol resolution
- taskToolProtocol: used for protocol resolution
- rooIgnoreController: used for access validation
- fileContextTracker: used for tracking file context

This fix ensures the test properly mocks all dependencies required by the
ReadFileTool when invoked through presentAssistantMessage.

…y.limit and truncatedByLimit

- Fix MAX_LINE_BYTES to 500 (was 2000), rename from MAX_LINE_LENGTH for clarity
- Implement byte-based UTF-8 line truncation at safe boundaries (safeUtf8Truncate)
- Rename FALLBACK_LIMIT to DEFAULT_LINE_LIMIT for clarity
- Add maxLines property to indentation schema in tool definition
- Add test for UTF-8 boundary truncation with emoji

…cluded